Skip to content

Control works c#3

Open
Andrew-Kochanov wants to merge 10 commits intomainfrom
controlWorksC
Open

Control works c#3
Andrew-Kochanov wants to merge 10 commits intomainfrom
controlWorksC

Conversation

@Andrew-Kochanov
Copy link
Owner

No description provided.

@Andrew-Kochanov
Copy link
Owner Author

Я уже понял, в чем проблема

Copy link
Collaborator

@WoWaster WoWaster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лично Вам -1 балл к обеим задачам за неумение следовать требованиям (здесь: прикладывание ссылки на HwProj)

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Удивительно всё хорошо. 10 баллов.

Comment on lines +28 to +32
if (flag) {
printf("Строка палиндром\n");
} else {
printf("Строка не палиндром\n");
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ну нет, это должно быть в main, а отсюда возвращаться true или false.

#include <stdio.h>
#include <string.h>

void palindrome(char* str)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

По стайлгайду вообще isPalindrome, но сойдёт.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

9 баллов, -1 получен за смешение логики и ввода-вывода.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants